Skip to content

Drop each complex without an ortholog under the drop policy - #25

Open
edkerk wants to merge 1 commit into
developfrom
feat/drop-per-complex
Open

edkerk wants to merge 1 commit into
developfrom
feat/drop-per-complex

Conversation

@edkerk

@edkerk edkerk commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

getModelFromHomology with complexPolicy drop removes each complex that has a subunit without an ortholog, and removes the reaction only when no complex remains. For the rule (A and B) or (C and D) without an ortholog of B, the draft has the orthologs of C and D. Before, the reaction was removed as soon as one subunit had no ortholog.

The step that removes the placeholder genes of isozymes reads the rule as a tree, in which and binds tighter than or. For the rule (A or B and C) and D without an ortholog of B, the draft rule is (nA or (OLD_T_B and nC)) and nD under flag, (nA or nC) and nD under keep and nA and nD under drop. The result differs from before only for rules with unbracketed and and or or with nested or groups.

The same policy is implemented in raven-toolbox#199.

The policy drop removes each complex that has a subunit without an ortholog and removes the reaction when no complex remains. It removed the reaction as soon as one subunit had no ortholog.
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Function test results

539 tests   529 ✅  2m 50s ⏱️
 24 suites    9 💤
  1 files      0 ❌  1 🔥

For more details on these errors, see this check.

Results for commit a037baa.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant